Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "Pin Message" Option in Reply-in-Thread Modal #704

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

smritidoneria
Copy link
Contributor

Brief Title

Acceptance Criteria fulfillment

-Implemented the "Pin Message" option in the "Reply in Thread" modal.

  • Ensured the pinned message functionality integrates seamlessly with the existing pinning mechanism in the main chat.

Fixes #703

Video/Screenshots

Screen.Recording.2024-12-21.at.1.44.53.PM.mov

PR Test Details

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-<pr_number> after approval. Contributors are requested to replace <pr_number> with the actual PR number.

@Spiral-Memory
Copy link
Collaborator

Hi @smritidoneria
Can you show it in the video on how it display this pinned message in the chat as well as pinned message sidebar?

@smritidoneria
Copy link
Contributor Author

Hey @Spiral-Memory, currently, the pinned message is not displayed in the pinning message sidebar due to an issue discussed in this pr, #654. Once this issue is resolved, the pinned message in the thread will also appear in the sidebar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing "Pin Message" Feature in Reply-in-Thread Modal
2 participants